Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow passing path as input opt #13

Merged
merged 2 commits into from
Jan 8, 2021
Merged

allow passing path as input opt #13

merged 2 commits into from
Jan 8, 2021

Conversation

staltz
Copy link
Member

@staltz staltz commented Jan 7, 2021

Context: ssbc/ssb-fixtures#2

Problem: I have generated an SSB database but I want different authors (different ssb keys objects) in scuttle-testbot to share the same folder. scuttle-testbot currently make the path from the name, but this forces us to use /tmp/something folders.

@staltz staltz requested review from mixmix and arj03 January 7, 2021 15:26
@staltz staltz closed this Jan 7, 2021
@staltz staltz reopened this Jan 7, 2021
@staltz
Copy link
Member Author

staltz commented Jan 7, 2021

Pushed a commit that makes the test simpler, should pass all tests and operating systems in Travis.

Copy link
Member

@mixmix mixmix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Happy for this to be merged and deployed.
Have been thinking about how this tool can be used for ssb-db and ssb-db2

@@ -47,7 +47,8 @@ By default, CreateTestSbot deletes an existing database of the same `name` befor

Valid `opts` keys include:
- `name` *String* (optional) (default: `ssb-test + Number(new Date)`)
- `myTestName`: Sets the database in /tmp/myTestName
- `path` *String* (optional) (default: `/tmp/${name}`, where `name` is the above)
- `~/.ssb-test`: Sets the database in `~/.ssb-test`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sets the working location for the identity - log databases, secret, conn.json etc

Wonder if there's an easy word for that.
Anyway not a big deal

@staltz
Copy link
Member Author

staltz commented Jan 8, 2021

Nice, can has merge?

@arj03 arj03 merged commit 5c2ccf6 into master Jan 8, 2021
@arj03
Copy link
Member

arj03 commented Jan 8, 2021

Checked the travis error, seems to have been failing before as well.

@arj03
Copy link
Member

arj03 commented Jan 8, 2021

Published as 1.5

@staltz staltz deleted the folderopt branch January 8, 2021 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants